-
Notifications
You must be signed in to change notification settings - Fork 156
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Python support for 3.13 #881
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #881 +/- ##
=====================================
Coverage 95.5% 95.6%
=====================================
Files 46 46
Lines 4340 4335 -5
=====================================
- Hits 4148 4147 -1
+ Misses 192 188 -4
|
7e9d42b
to
10b9e8d
Compare
10b9e8d
to
84899f8
Compare
Rebased with the completion of #544. If there are no obstacles, we can update this with parallel improvements to the others made on that branch. |
84899f8
to
9ba20da
Compare
9ba20da
to
89abcfc
Compare
- Use macos-13 for Python 3.9. - Use macos-latest for Python 3.10–3.13. - Parallel iiasa/ixmp#532
89abcfc
to
78b1291
Compare
Import of these from `typing` is deprecated from Python 3.9.
78b1291
to
19d20d1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@glatterf42 I leave this approval so that merging is un-blocked, but please let me know if we should discuss any of the later commits I added to the branch.
Thanks for finishing this, I don't think we need to discuss any commits here :) |
Please do the honours 🚀 |
As suggested by @khaeru in #880, we should drop support for the now outdated version 3.8 of Python and add support for the new 3.13. This PR does just that.
Please also see the analogous iiasa/ixmp#544, which details some issues that need to be closed upstream before we can support 3.13.
How to review
PR checklist